Skip to content

AssociationTableMixinClassReflectionExtension must rely on generic to detect native methods#57

Merged
rochamarcelo merged 7 commits into4.next-cake5from
feature/typed-association-find-by
Oct 15, 2025
Merged

AssociationTableMixinClassReflectionExtension must rely on generic to detect native methods#57
rochamarcelo merged 7 commits into4.next-cake5from
feature/typed-association-find-by

Conversation

@rochamarcelo
Copy link
Contributor

@rochamarcelo rochamarcelo commented Oct 14, 2025

Related to #55

@rochamarcelo
Copy link
Contributor Author

rochamarcelo commented Oct 14, 2025

@dereuromark Could you test this. Now it should not cause errors when using annotation

@property \Cake\ORM\Association\BelongsTo<\App\Model\Table\UsersTable> $Users

$this->Notes->Users->findByNamed('John Doe')->id// Native method to return User

$this->Notes->Users->findByRole('admin')->all()->toArray()//Magic finder method (not native method)

@dereuromark
Copy link
Collaborator

dereuromark commented Oct 14, 2025

Works great!

@rochamarcelo rochamarcelo merged commit c3e4fd7 into 4.next-cake5 Oct 15, 2025
6 checks passed
@dereuromark dereuromark deleted the feature/typed-association-find-by branch October 15, 2025 12:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@rochamarcelo @dereuromark